Clean some things up in the publishing article#55003
Conversation
There was a problem hiding this comment.
Pull request overview
This PR refines the .NET application publishing overview article by adjusting section naming, adding guidance to avoid common CLI publishing pitfalls, and simplifying/removing some pivoted content.
Changes:
- Add an IMPORTANT note warning about mixing build and publish options when using
dotnet publish --no-build. - Rename several publishing mode section headings to the “Publish as …” format.
- Adjust the “Quick reference” section text and pivot structure.
Comments suppressed due to low confidence (4)
docs/core/deploying/index.md:249
- Renaming this heading changes its autogenerated anchor. Other docs deep-link into this page using
#self-contained-deployment, so those links will break unless you keep the old anchor ID.
## Publish as self-contained
docs/core/deploying/index.md:298
- Renaming this heading changes its autogenerated anchor. The Quick reference table links to
#single-file-deployment, so it will break unless you keep the old anchor ID (or update all references).
## Publish as single-file
docs/core/deploying/index.md:351
- Renaming this heading changes its autogenerated anchor. The Quick reference table links to
#native-aot-deployment, so it will break unless you keep the old anchor ID (or update all references).
## Publish as native AOT
docs/core/deploying/index.md:413
- Renaming this heading changes its autogenerated anchor. The Quick reference table (and any external deep links) still use
#readytorun-deployment, so those links will break unless you keep the old anchor ID.
## Publish as ReadyToRun
There was a problem hiding this comment.
@adegeo, Looks good. Approved assuming current issues are addressed. I noted a couple of additional really minor items you could take or leave.
Co-authored-by: Wade Pickett <wpickett@microsoft.com>
There was a problem hiding this comment.
@adegeo: Approved, but see my minor fix suggestions inline. I am watching to keep this moving...
wadepickett
left a comment
There was a problem hiding this comment.
I'll watch for the link updates.
fa2a92f to
fa6171c
Compare
Summary
Related to the issue, I had already performed quite a large face lift on this content, merging articles together and covering the 3 tools, VS, VSCode, CLI. I instead focused on some section naming, calling out a trap people might find themselves in, and removing the pivots related to the CLI cheat sheet.
Fixes #47080
Internal previews
Toggle expand/collapse